-
Notifications
You must be signed in to change notification settings - Fork 408
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding pt-br translation #87
Conversation
lib/locales/pagy.yml
Outdated
prev: "‹ Anterior" | ||
next: "Próximo ›" | ||
gap: "…" | ||
current: "Esta na página" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Esta" should be "Está"
lib/locales/pagy.yml
Outdated
zero: "Sem resultados" | ||
one: "Mostrando 1 %{item_name}" | ||
other: "Mostrando %{count} %{item_name}" | ||
multiple_pages: "Mostrando %{item_name} %{from}-%{to} de %{count} no total" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be de other way around:
"Mostrando %{from}-%{to} %{item_name} de %{count} no total"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Mostrando %{from}-%{to} %{item_name} de %{count} no total"
myabe ? Mostrando %{item_name} %{from}-%{to} no total de %{count}
Thanks for this. Do you think it could be possible to get a single translation that could work for both pt-br and for pt-pt? |
i dont know! |
@Pompeu: According to @gamafranco something of what is in the pt-br would not sound right in pt-pt. My request was if you could find a common way to translate it, so it would work for pt-br and pt-pt. @gamafranco ? |
i dont know, i see all time in same libs, pt-br and pt-pt, i see this in subtitles of films too. |
lib/locales/pagy.yml
Outdated
of: "de" | ||
items: | ||
show: "Mostrar" | ||
items: "items por página" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
items: "items por página"
Should it be? 'itens por página'
lib/locales/pagy.yml
Outdated
of: "de" | ||
items: | ||
show: "Mostrar" | ||
items: "items por página" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could use other
or items_per_page
instead of items:
twice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will fix all "items" to itens
@Pompeu: yes, that is what I mean... however IIRC, in a private chat @gamafranco told me something about "Mostrar" that is not used in regular Portuguese. So what I was asking is translating the file in such a way that it would sound right for both Brazilian and Portuguese. |
The problem is not with "mostrar", it's with "mostrando" |
We use "a mostrar" instead of "mostrando" |
@Pompeu, @juuh42dias Would "a mostrar" work also for pt-br? |
|
@Pompeu @juuh42dias I suggest to post a quick round of alternative translation that work for pt-br... |
Added as pt-br; pt-pt will get another entry. |
No description provided.